-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Integrated player changes #175
base: dev
Are you sure you want to change the base?
Conversation
Issue with linting or unit tests detected.
For more information on our linting policies, please see our Linting-Guide. |
@@ -100,6 +100,13 @@ export const MODULE_MAP = { | |||
discovery: DISCOVERY_MODULE_MAP, | |||
}; | |||
|
|||
try { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add comment here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
@@ -102,4 +104,13 @@ export default class SetApiResponseHandler extends BaseHandler { | |||
const reportIdString = JSON.stringify({ report: 'Received UserInterest apiResponse parameters' }); | |||
return reportIdString; | |||
} | |||
|
|||
setExternalResponse(message) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
function description
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Description
Enter a detailed description of the PR here, including the nature and purpose of the changes.
JIRA Ticket
Include a link to the JIRA Ticket linked to this PR.
Issue-0000
Additional Info (Test steps / Steps to reproduce etc.)
Details on how to test the feature